MPT-21850 track shared CodeRabbit config#334
Merged
d3rky merged 1 commit intoJun 4, 2026
Merged
Conversation
Point CodeRabbit directly to the shared mpt-extension-skills configuration on main, matching the playground template update for MPT-21850.
📝 WalkthroughWalkthroughThe ChangesCodeRabbit Configuration Update
🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
🔕 Pre-merge checks override appliedThe pre-merge checks have been overridden successfully. You can now proceed with the merge. Overridden by ❌ Failed checks (1 error)
✅ Passed checks (3 passed)
Comment |
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.coderabbit.yaml:
- Line 3: The added URL in .coderabbit.yaml points to
mpt-extension-skills/coderabbit-shared.yaml which injects a pre-merge policy
requiring the exact first PR description line "🤖 AI-generated PR — Please
review carefully.", causing a merge blocker; either remove or replace the "url"
entry to avoid inheriting that strict policy (edit the url field in
.coderabbit.yaml), or alternatively update the PR description to start exactly
with "🤖 AI-generated PR — Please review carefully." so it satisfies the
inherited check.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro
Run ID: c3e6e851-2a1a-48dd-ab9c-d33d5977be16
📒 Files selected for processing (1)
.coderabbit.yaml
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: d3rky
Repo: softwareone-platform/mpt-api-python-client PR: 299
File: docs/testing.md:3-5
Timestamp: 2026-04-14T13:31:12.234Z
Learning: In the softwareone-platform/mpt-api-python-client repository, the shared testing standards file in the mpt-extension-skills repository is `standards/unittests.md` (not `standards/testing-standard.md`). The correct URL is https://github.com/softwareone-platform/mpt-extension-skills/blob/main/standards/unittests.md. Do not flag `docs/testing.md` for linking to `standards/unittests.md`.
📚 Learning: 2026-04-16T11:49:53.579Z
Learnt from: jentyk
Repo: softwareone-platform/mpt-api-python-client PR: 303
File: tests/e2e/helpdesk/chats/participants/conftest.py:25-31
Timestamp: 2026-04-16T11:49:53.579Z
Learning: In tests/e2e/helpdesk/chats/participants/conftest.py (E2E fixtures `created_chat_participant` and `async_created_chat_participant`), it’s acceptable to omit participant teardown/cleanup. The parent `created_chat` fixture performs cascade-deletion of the chat and all related records (including participants) during teardown, so missing participant-specific cleanup should not be flagged in code review.
Applied to files:
.coderabbit.yaml
🔀 Multi-repo context softwareone-platform/mpt-extension-skills
Findings
softwareone-platform/mpt-extension-skills
- Presence of shared CodeRabbit config: coderabbit-shared.yaml at repository root. [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml]
- Key values observed (relevant to the PR switching remote_config.url to this file):
- inheritance: true, early_access: true, enable_free_tier: true. [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml:1-40]
- auto_review.enabled: true and auto_incremental_review: true. [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml:80-100]
- pre_merge_checks includes an "AI Warning Line In PR Description" custom check that errors if the exact warning line
🤖 AI-generated PR — Please review carefully.is not the first line of the PR description. (This is a strict requirement that will be enforced if this shared config is used.) [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml:120-160] - high_level_summary instructions include extracting Jira keys and prepending "Closes MPT-XXXX" where applicable. [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml:40-80]
- Many tooling flags (ruff, yamllint, markdownlint, semgrep, etc.) enabled in tools section. [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml:160-260]
Implication: Pointing .coderabbit.yaml.remote_config.url to this file will cause repositories using it (e.g., the PR target repo) to inherit the above policies and automated checks — notably the strict pre-merge AI-warning-line check and auto_review behaviors. [::softwareone-platform/mpt-extension-skills::coderabbit-shared.yaml]
d3rky
deleted the
bugfix/MPT-21850/fix-coderabbit-configuration-api-client
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 AI-generated PR — Please review carefully.
What changed
.coderabbit.yamlto use the shared CodeRabbit configuration directly fromsoftwareone-platform/mpt-extension-skills.swo-extension-playground.Testing
.coderabbit.yamlsuccessfully with Ruby YAML.Closes MPT-21850
Changes
.coderabbit.yamlto reference shared CodeRabbit configuration directly fromsoftwareone-platform/mpt-extension-skillsrepository instead of the intermediateswo-extension-playgroundreferenceremote_config.urlfromhttps://raw.githubusercontent.com/softwareone-platform/swo-extension-playground/refs/heads/main/.coderabbit.yamltohttps://raw.githubusercontent.com/softwareone-platform/mpt-extension-skills/main/coderabbit-shared.yaml